-
Notifications
You must be signed in to change notification settings - Fork 321
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable Copier Gain feature for DMIC interface #9537
Conversation
Define common IPC4 Dmic blob structures Signed-off-by: Ievgen Ganakov <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but can you also explain when/how DMA control IPC is used. Thanks
60f3a5d
to
9babc61
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No showstoppers. I'd like @singalsu to review the new dmic blob addition so this is not conflicting with any other tool.
@@ -322,6 +322,9 @@ int copier_dai_create(struct comp_dev *dev, struct copier_data *cd, | |||
return -EINVAL; | |||
} | |||
dai.out_fmt = &copier->out_fmt; | |||
#if CONFIG_COPIER_GAIN | |||
dai.apply_gain = true; | |||
#endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happens when no configuration is passed from the host (like in case of Linux driver)? Unity gain applied? This seems to be enabled by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kv2019i Yes, unity gain applied (gain = 1)
Enable gain for DMIC interface. Configure gain feature with parameters received in DMIC BLOB. Add support for runtime DMIC gain parameters update using DMA Control IPC. Signed-off-by: Ievgen Ganakov <[email protected]>
9babc61
to
0fd9030
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks OK to me. The dmic structs are same though here more detailed based on updated documentation vs. the other dmic.h. It could be cleaned up later to avoid duplication.
Apply Copier Gain feature, introduced in #9323, to DMIC interface.
Add complete DMIC configuration blob definitions.
Use DMA Control IPC message to update gain coefficients in runtime.